Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updating incidence to merge in Nates edits #201

Merged
merged 3 commits into from
Oct 2, 2023
Merged

Conversation

jreps
Copy link
Collaborator

@jreps jreps commented Sep 23, 2023

  • updated component input selection to enable div or other UIs that don't have an input
  • update input selection to add a reset function (use updateFunction to specify the update function name)
  • added Nate's formatting to add selected options on new lines
  • updated Nate's incidence rate module to use standard input selection

- updated component input selection to enable div or other UIs that don't have an input
- update input selection to add a reset function (use updateFunction to specify the update function name)
- added Nate's formatting to add selected options on new lines
- updated Nate's incidence rate module to use standard input selection
@jreps jreps requested review from azimov and nhall6 September 23, 2023 23:11
if (cur_dev > 1) grDevices::dev.set(cur_dev)
})
# Convert to ggplot-------------
cowplot::ggdraw() +
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just confirming that this package is added to develop?

Adding it was necessary as pulling the legend out of the plot and placing it below was very difficult without this.

@azimov
Copy link
Collaborator

azimov commented Sep 26, 2023

Just a few build warnings and 1 failed test, otherwise this looks good

fixing incidence rate tar bug
fixing R check
@codecov
Copy link

codecov bot commented Oct 2, 2023

Codecov Report

Attention: 294 lines in your changes are missing coverage. Please review.

Comparison is base (e7589b8) 79.35% compared to head (b7f8e80) 78.56%.
Report is 4 commits behind head on develop.

❗ Current head b7f8e80 differs from pull request most recent head 2989f9f. Consider uploading reports for the commit 2989f9f to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #201      +/-   ##
===========================================
- Coverage    79.35%   78.56%   -0.80%     
===========================================
  Files           72       72              
  Lines        17135    17803     +668     
===========================================
+ Hits         13598    13987     +389     
- Misses        3537     3816     +279     
Files Coverage Δ
R/cohort-diagnostics-characterization.R 95.91% <100.00%> (ø)
R/cohort-diagnostics-main-ui.R 100.00% <100.00%> (ø)
R/cohort-diagnostics-main.R 86.97% <100.00%> (ø)
R/helpers-sccsDataPulls.R 100.00% <ø> (ø)
R/helpers-sccsPlots.R 91.18% <100.00%> (+0.69%) ⬆️
R/patient-level-prediction-designSummary.R 97.45% <100.00%> (+0.02%) ⬆️
R/sccs-results-full.R 34.50% <25.00%> (-0.28%) ⬇️
R/components-inputselection.R 72.72% <42.30%> (-7.08%) ⬇️
R/characterization-incidence.R 64.01% <62.39%> (-29.24%) ⬇️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jreps jreps merged commit 6e7859a into develop Oct 2, 2023
6 of 8 checks passed
@jreps jreps deleted the incidence_merge branch October 2, 2023 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants